Conversation
|
Thanks for the contribution! Please review the labels and make any necessary changes. |
Summary of ChangesHello @taegeon2, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 Figma 디자인 시스템에 맞춰 Small, Medium, Large 세 가지 크기의 버튼 컴포넌트를 구현하여 재사용 가능한 UI 요소를 제공합니다. 이를 통해 애플리케이션 전반에 걸쳐 일관된 사용자 경험을 유지하고 개발 효율성을 높일 수 있습니다. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
| className={` | ||
| w-[21.5625rem] py-[0.6875rem] flex flex-col items-center rounded-[0.625rem] border-[0.5px] | ||
| ${ | ||
| isSelected | ||
| ? "border-[var(--color-primary)] bg-[var(--color-primary-tint)]" | ||
| : "border-[var(--color-content-secondary)] bg-[var(--color-neutral)]" | ||
| } | ||
| `} |
There was a problem hiding this comment.
조건부 Tailwind CSS 클래스를 사용하는 className 문자열이 길어져 가독성이 저하될 수 있습니다. clsx와 같은 유틸리티를 사용하면 조건부 클래스를 더 깔끔하게 관리하고 가독성을 높일 수 있습니다.
| className={` | |
| w-[21.5625rem] py-[0.6875rem] flex flex-col items-center rounded-[0.625rem] border-[0.5px] | |
| ${ | |
| isSelected | |
| ? "border-[var(--color-primary)] bg-[var(--color-primary-tint)]" | |
| : "border-[var(--color-content-secondary)] bg-[var(--color-neutral)]" | |
| } | |
| `} | |
| className={clsx( | |
| "w-[21.5625rem] py-[0.6875rem] flex flex-col items-center rounded-[0.625rem] border-[0.5px]", | |
| isSelected | |
| ? "border-[var(--color-primary)] bg-[var(--color-primary-tint)]" | |
| : "border-[var(--color-content-secondary)] bg-[var(--color-neutral)]" | |
| )} |
| //PressableProps: 버튼 컴포넌트의 속성을 정의 | ||
| children: ReactNode; // ReactNode: 모든 타입의 자식 요소를 허용 |
src/shared/ui/buttons/buttons.md
Outdated
|
|
||
| ## LargeButton | ||
|
|
||
| 제목 + 설명을 포함하는 토글형 대형 버튼입니다. 클릭 시 선택/해제 상태가 전환됩니다. |
There was a problem hiding this comment.
large_button의 경우 모바일에서 클릭 시 색이 변한다는 것을 의미였습니다!
|
태건님, 컴포넌트 작업해주시느라 정말 고생하셨습니다! 깔끔하게 잘 만들어주셨네요. :) 작업하신 내용 보면서 같이 논의해보면 좋을 포인트가 두 개 정도 있어서 남겨드려요. 파일 이름) SelectionGroup 컴포넌트) 이런 느낌으로 하면 더 간편하게 사용 가능할 것 같다는 생각이 들었어요 보시고 편하게 의견 주세요! |
좋은 의견 감사합니다!
부족한 부분 지적 감사합니다! |
📝 요약
⚙️ 작업 내용
🔗 관련 이슈
✅ 체크리스트
💬 리뷰어에게